Skip to content

Fix: Correct Spack capitalization in tutorial files.#439

Merged
haampie merged 4 commits into
spack:mainfrom
haampie:fix-spack-capitalization
Jun 3, 2025
Merged

Fix: Correct Spack capitalization in tutorial files.#439
haampie merged 4 commits into
spack:mainfrom
haampie:fix-spack-capitalization

Conversation

@haampie

@haampie haampie commented Jun 3, 2025

Copy link
Copy Markdown
Member

This commit corrects the capitalization of "Spack" in all tutorial_*.rst files.

The changes ensure that:

  • "Spack" is used when referring to the project or tool by name.
  • "spack" is used when referring to the command-line executable.

This commit corrects the capitalization of "Spack" in all tutorial_*.rst files.

The changes ensure that:
- "Spack" is used when referring to the project or tool by name.
- "spack" is used when referring to the command-line executable.
Comment thread tutorial_buildsystems.rst Outdated
Comment thread tutorial_configuration.rst Outdated
Comment thread tutorial_developer_workflows.rst Outdated
Comment thread tutorial_environments.rst Outdated
Comment thread tutorial_modules.rst Outdated
Comment thread tutorial_modules.rst Outdated
Comment thread tutorial_packaging.rst Outdated

@haampie haampie left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some unrelated changes

Comment thread tutorial_configuration.rst Outdated
@haampie haampie requested a review from Copilot June 3, 2025 15:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR standardizes the capitalization of “Spack” (tool name) versus “spack” (CLI executable) across all tutorial_*.rst files to improve consistency.

  • Ensures “Spack” is used when referring to the project/tool
  • Keeps “spack” lowercase when denoting the command-line executable
  • Adjusts the example setup script path to align with the revised placeholder

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
tutorial_developer_workflows.rst Updated environment references to “Spack” and wrapped commands in backticks
tutorial_configuration.rst Corrected “Spack” capitalization in links, list items, and descriptions
tutorial_buildsystems.rst Adjusted the placeholder path for the Spack setup script
tutorial_advanced_packaging.rst Fixed “Spack” capitalization in a build-time environment example

@haampie haampie merged commit 15094b0 into spack:main Jun 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants